-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Add controls feature to tweak the visualizer easier #1180
feat: Add controls feature to tweak the visualizer easier #1180
Conversation
…isualizer/randomize-amplitude
…a-visualizer/add-emitter-tilting
Signed-off-by: Eugene Panteleymonchuk <[email protected]>
Signed-off-by: Eugene Panteleymonchuk <[email protected]>
…-tilting' into feat/issues-1171-add-controls-for-visualizer # Conflicts: # client/src/features/visualizer-threejs/CameraControls.tsx
Signed-off-by: Eugene Panteleymonchuk <[email protected]>
Signed-off-by: Eugene Panteleymonchuk <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some suggestions
…nfigControls.tsx Co-authored-by: JCNoguera <[email protected]>
…nfigControls.tsx Co-authored-by: JCNoguera <[email protected]>
Signed-off-by: Eugene Panteleymonchuk <[email protected]>
…-tilting' into feat/issues-1171-add-controls-for-visualizer # Conflicts: # client/src/features/visualizer-threejs/utils.ts
a6991c3
to
cba1854
Compare
…-tilting' into feat/issues-1171-add-controls-for-visualizer
Signed-off-by: Eugene Panteleymonchuk <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few more changes
Signed-off-by: Eugene Panteleymonchuk <[email protected]>
Signed-off-by: Eugene Panteleymonchuk <[email protected]>
…ontrols-for-visualizer
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just solve conflicts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description of change
closes #1171
How the change has been tested
Enable feature
controlsVisualiserEnabled
inVisualizerInstance
. Select value that need to be changed and press "apply".Expected result:
Visualizer uses sat values.
Change checklist
Add an
x
to the boxes that are relevant to your changes, and delete any items that are not.